Skip to content

update docs#83

Merged
EzicStar merged 2 commits intomainfrom
docs/encoding
Jul 1, 2025
Merged

update docs#83
EzicStar merged 2 commits intomainfrom
docs/encoding

Conversation

@EzicStar
Copy link
Copy Markdown
Contributor

@EzicStar EzicStar commented Jul 1, 2025

  • add tutorial on how to use slide and patient encoding
  • set chief as default encoder
  • update readme for new installation steps with uv

- add tutorial on how to use slide and patient encoding
- set chief as default encoder
- update readme for new installation steps with uv
@EzicStar EzicStar requested a review from s1787956 July 1, 2025 09:44
@EzicStar EzicStar self-assigned this Jul 1, 2025
Copy link
Copy Markdown
Contributor

@s1787956 s1787956 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice work. I think there is a cd STAMP missing but otherwise its all good

Comment thread README.md
Comment thread getting-started.md
| COBRA2 | CONCH, UNI, VIRCHOW2 or H-OPTIMUS-0 | CUDA only
| EAGLE | CTRANSPATH, CHIEF-CTRANSPATH | CUDA only
| MADELEINE | CONCH | CUDA only

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

crazy, I didnt know so many are cuda only? I thought only mamba models were affected Oo

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there are some like CHIEF which I think that should be easy to accept non cuda devices, but as soon as I read in their github installation guides that they were tested on CUDA and they did not work on the pipeline, i just left it like that. Maybe we should add an issue to debug each model on a mac and linux without cuda to see if we can make them compatible.

add cd STAMP in installation steps
@EzicStar EzicStar closed this Jul 1, 2025
@EzicStar EzicStar reopened this Jul 1, 2025
@EzicStar EzicStar merged commit f09407f into main Jul 1, 2025
55 checks passed
@EzicStar EzicStar deleted the docs/encoding branch August 20, 2025 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants